Fix and update stress tests for Event Hubs and Service Bus#49435
Merged
Conversation
… even if Application Insights is not available. Could be other listeners.
conniey
requested review from
a team,
EldertGrootenboer,
SwayGom,
axisc,
hmlam,
j7nw4r,
sagar0207,
sjkwak and
skarri-microsoft
as code owners
Contributor
There was a problem hiding this comment.
Pull request overview
This PR updates the Java Event Hubs and Service Bus stress-test packages to align with EngSys stress-test framework expectations, remove the legacy useV2Stack toggle, and simplify image build/deploy workflows.
Changes:
- Removes the
useV2Stack/USE_V2configuration across Event Hubs stress code and Helm templates. - Refactors stress image builds to rely on pre-built artifacts (new
New-StressTestRun.ps1scripts + simplifiedDockerfiles) and updates scenarios matrix paths. - Updates stress-test documentation and Helm chart dependencies (
stress-test-addonsrepo/version) and telemetry initialization behavior.
Reviewed changes
Copilot reviewed 15 out of 15 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| sdk/servicebus/azure-messaging-servicebus-stress/src/main/java/com/azure/messaging/servicebus/stress/util/TelemetryHelper.java | Adjusts OpenTelemetry autoconfiguration/exporter behavior and enables GC cause capture. |
| sdk/servicebus/azure-messaging-servicebus-stress/scenarios-matrix.yaml | Updates imageBuildDir to point at the package directory. |
| sdk/servicebus/azure-messaging-servicebus-stress/README.md | Refreshes deployment/monitoring/troubleshooting guidance and project links. |
| sdk/servicebus/azure-messaging-servicebus-stress/New-StressTestRun.ps1 | Adds a helper script to build (optionally) + login (optionally) + deploy stress tests. |
| sdk/servicebus/azure-messaging-servicebus-stress/Dockerfile | Switches to copying pre-built artifacts into the runtime image (no in-Docker build). |
| sdk/servicebus/azure-messaging-servicebus-stress/Chart.yaml | Updates stress-test-addons dependency repo alias and version range. |
| sdk/eventhubs/azure-messaging-eventhubs-stress/templates/job.yaml | Removes node selector and USE_V2 export from sender/receiver containers. |
| sdk/eventhubs/azure-messaging-eventhubs-stress/src/main/java/com/azure/messaging/eventhubs/stress/util/TestUtils.java | Removes v2-stack configuration injection into Event Hubs builders. |
| sdk/eventhubs/azure-messaging-eventhubs-stress/src/main/java/com/azure/messaging/eventhubs/stress/util/TelemetryHelper.java | Updates OpenTelemetry autoconfiguration/exporter behavior and removes useV2 span attribute. |
| sdk/eventhubs/azure-messaging-eventhubs-stress/src/main/java/com/azure/messaging/eventhubs/stress/util/ScenarioOptions.java | Removes USE_V2 option/property and accessor. |
| sdk/eventhubs/azure-messaging-eventhubs-stress/src/main/java/com/azure/messaging/eventhubs/stress/scenarios/EventForwarder.java | Changes forwarder producer auth to DefaultAzureCredential + FQNS, removing forward connection string usage. |
| sdk/eventhubs/azure-messaging-eventhubs-stress/scenarios-matrix.yaml | Updates imageBuildDir and removes the useV2 matrix dimension. |
| sdk/eventhubs/azure-messaging-eventhubs-stress/README.md | Refreshes deployment/monitoring instructions and example scenarios; adds reference links. |
| sdk/eventhubs/azure-messaging-eventhubs-stress/New-StressTestRun.ps1 | Adds a helper script to build (optionally) + login (optionally) + deploy stress tests. |
| sdk/eventhubs/azure-messaging-eventhubs-stress/Dockerfile | Switches to copying pre-built artifacts into the runtime image (no in-Docker build). |
anuchandy
approved these changes
Jun 10, 2026
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
anuchandy
approved these changes
Jun 10, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
This updates the Event Hubs and Service Bus stress tests.
All SDK Contribution checklist:
General Guidelines and Best Practices
Testing Guidelines